Skip to content

Conversation

@shubhjha123
Copy link

In our use of gripmock we came across a need of extra delay over grpc response. Added code for extraa delay for grpc apis, which can help library users to configure any delay in their api response.

Ardiannn08
Ardiannn08 previously approved these changes Apr 13, 2021
@Ardiannn08 Ardiannn08 dismissed their stale review April 13, 2021 02:15

accidentally hit approve

@shubhjha123 shubhjha123 requested a review from Ardiannn08 April 14, 2021 04:51
@shubhjha123 shubhjha123 requested a review from Ardiannn08 April 18, 2021 09:53
Ardiannn08
Ardiannn08 approved these changes Apr 20, 2021
@shubhjha123
Copy link
Author

shubhjha123 commented Apr 21, 2021

Can we merge this pr? Ardiannn08 approved it.

}

type Output struct {
Delay Duration `json:"delay"`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can't do it directly under Output struct, can you please create a Meta field and put duration under it?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @jekiapp Duration is meta field defined in file stub/duration.go, such that
type Duration struct {
time.Duration
}

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@shubhjha123 I mean like this

type Output struct{
  Meta Meta
}

type Meta struct{
  Delay Duration
}

@jekiapp
Copy link
Contributor

jekiapp commented May 6, 2021

Hi, sorry for the delayed review 🙏 I just saw it today. if you're nakama you can reach me by @jeki in slack.

@shubhjha123 shubhjha123 requested a review from jekiapp October 3, 2021 13:31
@yogesh1608
Copy link

@jekiapp @shubhjha123 @Ardiannn08 is somebody looking to merge this? This seems to be a need now.

@akuteRoblo
Copy link

What is the status on this?

@yzhuang93
Copy link

Is this feat still planned to be merged? Adding delay is a very useful feature for mocking several real test cases, good to have this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants